[INFO] cloning repository https://github.com/Kwarf/Allie-2017
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/Kwarf/Allie-2017" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKwarf%2FAllie-2017", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKwarf%2FAllie-2017'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] bd2a1bc2b49e2c6bb8a7d7b20602219ef7f2b724
[INFO] checking Kwarf/Allie-2017 against master#03c609abb6638f9d7f49f34326d4137d07f5cd61 for pr-155945
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKwarf%2FAllie-2017" "/workspace/builds/worker-4-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/Kwarf/Allie-2017
[INFO] finished tweaking git repo https://github.com/Kwarf/Allie-2017
[INFO] tweaked toml for git repo https://github.com/Kwarf/Allie-2017 written to /workspace/builds/worker-4-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/Kwarf/Allie-2017 on toolchain 03c609abb6638f9d7f49f34326d4137d07f5cd61
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+03c609abb6638f9d7f49f34326d4137d07f5cd61" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/Kwarf/Allie-2017 already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+03c609abb6638f9d7f49f34326d4137d07f5cd61" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded num-traits v0.1.37
[INFO] [stderr]   Downloaded serde_json v0.9.9
[INFO] [stderr]   Downloaded syn v0.11.9
[INFO] [stderr]   Downloaded serde v0.9.11
[INFO] [stderr]   Downloaded either v1.1.0
[INFO] [stderr]   Downloaded vec_map v0.7.0
[INFO] [stderr]   Downloaded bitflags v0.8.0
[INFO] [stderr]   Downloaded atty v0.2.2
[INFO] [stderr]   Downloaded serde_codegen_internals v0.14.1
[INFO] [stderr]   Downloaded itertools v0.5.9
[INFO] [stderr]   Downloaded unicode-segmentation v1.1.0
[INFO] [stderr]   Downloaded libc v0.2.21
[INFO] [stderr]   Downloaded serde_derive v0.9.11
[INFO] [stderr]   Downloaded clap v2.21.1
[INFO] [stderr]   Downloaded pathfinding v0.1.10
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+03c609abb6638f9d7f49f34326d4137d07f5cd61" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 80549b1835657f0c6136852d6ce676df06a76ebe15d0674d1daef1f057708421
[INFO] running `Command { std: "docker" "start" "-a" "80549b1835657f0c6136852d6ce676df06a76ebe15d0674d1daef1f057708421", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "80549b1835657f0c6136852d6ce676df06a76ebe15d0674d1daef1f057708421", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "80549b1835657f0c6136852d6ce676df06a76ebe15d0674d1daef1f057708421", kill_on_drop: false }`
[INFO] [stdout] 80549b1835657f0c6136852d6ce676df06a76ebe15d0674d1daef1f057708421
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+03c609abb6638f9d7f49f34326d4137d07f5cd61" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 2b428e84811000f370628706b63da884615f95db6a658c2e0a1e407eee8f3b93
[INFO] running `Command { std: "docker" "start" "-a" "2b428e84811000f370628706b63da884615f95db6a658c2e0a1e407eee8f3b93", kill_on_drop: false }`
[INFO] [stderr] warning: `package.edition` is unspecified, defaulting to `2015` while the latest is `2024`
[INFO] [stderr]     Checking libc v0.2.21
[INFO] [stderr]     Checking num-traits v0.1.37
[INFO] [stderr]     Checking serde v0.9.11
[INFO] [stderr]    Compiling syn v0.11.9
[INFO] [stderr]     Checking ansi_term v0.9.0
[INFO] [stderr]     Checking unicode-segmentation v1.1.0
[INFO] [stderr]     Checking strsim v0.6.0
[INFO] [stderr]     Checking bitflags v0.8.0
[INFO] [stderr]     Checking unicode-width v0.1.4
[INFO] [stderr]     Checking either v1.1.0
[INFO] [stderr]     Checking vec_map v0.7.0
[INFO] [stderr]     Checking itertools v0.5.9
[INFO] [stderr]     Checking term_size v0.2.3
[INFO] [stderr]     Checking atty v0.2.2
[INFO] [stderr]     Checking clap v2.21.1
[INFO] [stderr]     Checking pathfinding v0.1.10
[INFO] [stderr]    Compiling serde_codegen_internals v0.14.1
[INFO] [stderr]    Compiling serde_derive v0.9.11
[INFO] [stderr]     Checking serde_json v0.9.9
[INFO] [stderr]     Checking allie v1.2.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `Instant`
[INFO] [stdout]   --> src/main.rs:15:27
[INFO] [stdout]    |
[INFO] [stdout] 15 | use std::time::{Duration, Instant};
[INFO] [stdout]    |                           ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `self::killer::Killer`
[INFO] [stdout]   --> src/ai/strategies/mod.rs:12:9
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub use self::killer::Killer;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated
[INFO] [stdout]   --> src/ai/mod.rs:16:33
[INFO] [stdout]    |
[INFO] [stdout] 16 |     strategies: Vec<RefCell<Box<Strategy>>>,
[INFO] [stdout]    |                                 ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021!
[INFO] [stdout]    = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/warnings-promoted-to-error.html>
[INFO] [stdout]    = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn`
[INFO] [stdout]    |
[INFO] [stdout] 16 |     strategies: Vec<RefCell<Box<dyn Strategy>>>,
[INFO] [stdout]    |                                 +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Instant`
[INFO] [stdout]   --> src/main.rs:15:27
[INFO] [stdout]    |
[INFO] [stdout] 15 | use std::time::{Duration, Instant};
[INFO] [stdout]    |                           ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `self::killer::Killer`
[INFO] [stdout]   --> src/ai/strategies/mod.rs:12:9
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub use self::killer::Killer;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated
[INFO] [stdout]   --> src/ai/mod.rs:16:33
[INFO] [stdout]    |
[INFO] [stdout] 16 |     strategies: Vec<RefCell<Box<Strategy>>>,
[INFO] [stdout]    |                                 ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = warning: this is accepted in the current edition (Rust 2015) but is a hard error in Rust 2021!
[INFO] [stdout]    = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/warnings-promoted-to-error.html>
[INFO] [stdout]    = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn`
[INFO] [stdout]    |
[INFO] [stdout] 16 |     strategies: Vec<RefCell<Box<dyn Strategy>>>,
[INFO] [stdout]    |                                 +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `duration_in_ms` is never used
[INFO] [stdout]    --> src/main.rs:111:4
[INFO] [stdout]     |
[INFO] [stdout] 111 | fn duration_in_ms(duration: &Duration) -> f32 {
[INFO] [stdout]     |    ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variant `Killer` is never constructed
[INFO] [stdout]   --> src/ai/strategies/mod.rs:19:5
[INFO] [stdout]    |
[INFO] [stdout] 16 | pub enum StrategyType {
[INFO] [stdout]    |          ------------ variant in this enum
[INFO] [stdout] ...
[INFO] [stdout] 19 |     Killer,
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `StrategyType` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `Killer` is never constructed
[INFO] [stdout]   --> src/ai/strategies/killer.rs:10:12
[INFO] [stdout]    |
[INFO] [stdout] 10 | pub struct Killer;
[INFO] [stdout]    |            ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new` and `shortest_enemy_path_cost` are never used
[INFO] [stdout]   --> src/ai/strategies/killer.rs:13:12
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl Killer {
[INFO] [stdout]    | ----------- associated functions in this implementation
[INFO] [stdout] 13 |     pub fn new() -> Killer {
[INFO] [stdout]    |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 17 |     fn shortest_enemy_path_cost(map: &Map, enemies: &[Player], to: &Position) -> usize {
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_enemy_nearby` is never used
[INFO] [stdout]   --> src/ai/strategies/pick_pellets.rs:21:8
[INFO] [stdout]    |
[INFO] [stdout] 14 | impl PickPellets {
[INFO] [stdout]    | ---------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 21 |     fn is_enemy_nearby(&self, bot: &Bot, state: &GameState) -> bool {
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_super_pellet` is never used
[INFO] [stdout]   --> src/game/mod.rs:32:12
[INFO] [stdout]    |
[INFO] [stdout] 17 | impl TileType {
[INFO] [stdout]    | ------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 32 |     pub fn is_super_pellet(&self) -> bool {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `neighbours`, `pellets`, `super_pellets`, and `index_to_position` are never used
[INFO] [stdout]   --> src/game/mod.rs:56:12
[INFO] [stdout]    |
[INFO] [stdout] 47 | impl Map {
[INFO] [stdout]    | -------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 56 |     pub fn neighbours(&self, position: &Position) -> Vec<(Direction, TileType)> {
[INFO] [stdout]    |            ^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 71 |     pub fn pellets(&self) -> HashSet<Position> {
[INFO] [stdout]    |            ^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 81 |     pub fn super_pellets(&self) -> HashSet<Position> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 91 |     fn index_to_position(&self, index: usize) -> Position {
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `walkable_positions`, `closest_turning_points`, and `dead_ends` are never used
[INFO] [stdout]    --> src/game/mod.rs:207:12
[INFO] [stdout]     |
[INFO] [stdout] 132 | impl MapInformation {
[INFO] [stdout]     | ------------------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 207 |     pub fn walkable_positions(&self) -> &HashSet<Position> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 211 |     pub fn closest_turning_points<T: HasDimensions>(&self, limits: &T, position: &Position) -> HashSet<Position> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 232 |     pub fn dead_ends(&self) -> &HashSet<Position> {
[INFO] [stdout]     |            ^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `id` and `score` are never read
[INFO] [stdout]   --> src/protocol/mod.rs:24:5
[INFO] [stdout]    |
[INFO] [stdout] 23 | pub struct Player {
[INFO] [stdout]    |            ------ fields in this struct
[INFO] [stdout] 24 |     id: u32,
[INFO] [stdout]    |     ^^
[INFO] [stdout] ...
[INFO] [stdout] 30 |     score: u32,
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Player` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `0` is never read
[INFO] [stdout]   --> src/protocol/mod.rs:57:26
[INFO] [stdout]    |
[INFO] [stdout] 57 |     DeserializationError(serde_json::error::Error),
[INFO] [stdout]    |     -------------------- ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Error` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
[INFO] [stdout]    |
[INFO] [stdout] 57 -     DeserializationError(serde_json::error::Error),
[INFO] [stdout] 57 +     DeserializationError(()),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: enum `MessageType` is never used
[INFO] [stdout]  --> src/protocol/message_type.rs:4:10
[INFO] [stdout]   |
[INFO] [stdout] 4 | pub enum MessageType {
[INFO] [stdout]   |          ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: enum `MessageTypeError` is never used
[INFO] [stdout]   --> src/protocol/message_type.rs:13:10
[INFO] [stdout]    |
[INFO] [stdout] 13 | pub enum MessageTypeError {
[INFO] [stdout]    |          ^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/game/mod.rs:40:17
[INFO] [stdout]    |
[INFO] [stdout] 40 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Map` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Map`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variant `Killer` is never constructed
[INFO] [stdout]   --> src/ai/strategies/mod.rs:19:5
[INFO] [stdout]    |
[INFO] [stdout] 16 | pub enum StrategyType {
[INFO] [stdout]    |          ------------ variant in this enum
[INFO] [stdout] ...
[INFO] [stdout] 19 |     Killer,
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `StrategyType` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/mod.rs:10:17
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `GameState` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_GameState`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `Killer` is never constructed
[INFO] [stdout]   --> src/ai/strategies/killer.rs:10:12
[INFO] [stdout]    |
[INFO] [stdout] 10 | pub struct Killer;
[INFO] [stdout]    |            ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/mod.rs:22:17
[INFO] [stdout]    |
[INFO] [stdout] 22 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Player` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Player`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new` and `shortest_enemy_path_cost` are never used
[INFO] [stdout]   --> src/ai/strategies/killer.rs:13:12
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl Killer {
[INFO] [stdout]    | ----------- associated functions in this implementation
[INFO] [stdout] 13 |     pub fn new() -> Killer {
[INFO] [stdout]    |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 17 |     fn shortest_enemy_path_cost(map: &Map, enemies: &[Player], to: &Position) -> usize {
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_enemy_nearby` is never used
[INFO] [stdout]   --> src/ai/strategies/pick_pellets.rs:21:8
[INFO] [stdout]    |
[INFO] [stdout] 14 | impl PickPellets {
[INFO] [stdout]    | ---------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 21 |     fn is_enemy_nearby(&self, bot: &Bot, state: &GameState) -> bool {
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `is_super_pellet` is never used
[INFO] [stdout]   --> src/game/mod.rs:32:12
[INFO] [stdout]    |
[INFO] [stdout] 17 | impl TileType {
[INFO] [stdout]    | ------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 32 |     pub fn is_super_pellet(&self) -> bool {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `neighbours` and `super_pellets` are never used
[INFO] [stdout]   --> src/game/mod.rs:56:12
[INFO] [stdout]    |
[INFO] [stdout] 47 | impl Map {
[INFO] [stdout]    | -------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 56 |     pub fn neighbours(&self, position: &Position) -> Vec<(Direction, TileType)> {
[INFO] [stdout]    |            ^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 81 |     pub fn super_pellets(&self) -> HashSet<Position> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/protocol/json.rs:8:17
[INFO] [stdout]   |
[INFO] [stdout] 8 | #[derive(Debug, Deserialize)]
[INFO] [stdout]   |                 ^^^^^^^^^^^
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `BaseMessage` is not local
[INFO] [stdout]   |                 `Deserialize` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_BaseMessage`
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `walkable_positions` and `closest_turning_points` are never used
[INFO] [stdout]    --> src/game/mod.rs:207:12
[INFO] [stdout]     |
[INFO] [stdout] 132 | impl MapInformation {
[INFO] [stdout]     | ------------------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 207 |     pub fn walkable_positions(&self) -> &HashSet<Position> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 211 |     pub fn closest_turning_points<T: HasDimensions>(&self, limits: &T, position: &Position) -> HashSet<Position> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `0` is never read
[INFO] [stdout]   --> src/protocol/mod.rs:57:26
[INFO] [stdout]    |
[INFO] [stdout] 57 |     DeserializationError(serde_json::error::Error),
[INFO] [stdout]    |     -------------------- ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Error` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
[INFO] [stdout]    |
[INFO] [stdout] 57 -     DeserializationError(serde_json::error::Error),
[INFO] [stdout] 57 +     DeserializationError(()),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/json.rs:13:10
[INFO] [stdout]    |
[INFO] [stdout] 13 | #[derive(Deserialize)]
[INFO] [stdout]    |          ^^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `StateUpdateMessage` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_StateUpdateMessage`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/game/mod.rs:40:17
[INFO] [stdout]    |
[INFO] [stdout] 40 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Map` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Map`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/mod.rs:10:17
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `GameState` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_GameState`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/mod.rs:22:17
[INFO] [stdout]    |
[INFO] [stdout] 22 | #[derive(Debug, Deserialize)]
[INFO] [stdout]    |                 ^^^^^^^^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 `Player` is not local
[INFO] [stdout]    |                 `Deserialize` is not local
[INFO] [stdout]    |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Player`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/protocol/json.rs:8:17
[INFO] [stdout]   |
[INFO] [stdout] 8 | #[derive(Debug, Deserialize)]
[INFO] [stdout]   |                 ^^^^^^^^^^^
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `BaseMessage` is not local
[INFO] [stdout]   |                 `Deserialize` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_BaseMessage`
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/protocol/json.rs:13:10
[INFO] [stdout]    |
[INFO] [stdout] 13 | #[derive(Deserialize)]
[INFO] [stdout]    |          ^^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `StateUpdateMessage` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_StateUpdateMessage`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 5.53s
[INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: bitflags v0.8.0
[INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 4`
[INFO] running `Command { std: "docker" "inspect" "2b428e84811000f370628706b63da884615f95db6a658c2e0a1e407eee8f3b93", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "2b428e84811000f370628706b63da884615f95db6a658c2e0a1e407eee8f3b93", kill_on_drop: false }`
[INFO] [stdout] 2b428e84811000f370628706b63da884615f95db6a658c2e0a1e407eee8f3b93
